improve: fix pyproject.toml duplicate license entries and enhance AGENTS.md#37
improve: fix pyproject.toml duplicate license entries and enhance AGENTS.md#37Coding-Dev-Tools wants to merge 9 commits into
Conversation
🤖 Automated Code Review✅ Ruff Lint — No issues✅ Ruff Format — Clean🔴 Secret Detection — 605 potential secret(s)✅ Large Files — Within limits📊 Diff Stats — 16 file(s) changedVerdict: 🔴 Changes Requested — Potential secrets detected. Do not merge. Automated by Coding-Dev-Tools/.github reusable workflow. |
- Add tomli>=2.0 as conditional dependency for Python < 3.11 - Fix test_edge_cases.py to use try/except import for tomllib/tomli - This fixes CI failures on Python 3.10 where tomllib is not in stdlib
Coding-Dev-Tools
left a comment
There was a problem hiding this comment.
Auto-review: CI passes but secret-scan is noisy on the baseline. Please: 1) move any generated baseline off the PR diff, 2) add .secrets.baseline.new to .gitignore, 3) normalise test literal strings. No code changes from reviewer-A this round.
Coding-Dev-Tools
left a comment
There was a problem hiding this comment.
Reviewer-A checked the diff. Security concern: .secrets.baseline is committed and .secrets.baseline.new is in-tree. Secret-scan baselines generally should not be versioned alongside source; at minimum, ensure the baseline file is not tracking live secrets and .secrets.baseline.new is ignored. Also please trim diff churn from unrelated formatting baseline changes so core fix is reviewable.
|
Closing: The key fix (remove duplicate license entry in pyproject.toml) has been applied directly to main. The AGENTS.md improvement will be addressed separately. Generated by Claude Code |
Automated improvement by dev-engineer
Changes
These changes improve the project configuration and documentation structure.